Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add simple processing element for the switch demo #719

Open
wants to merge 12 commits into
base: staging
Choose a base branch
from

Conversation

hiddemoll
Copy link
Contributor

No description provided.

@hiddemoll hiddemoll force-pushed the hidde/switch-demo-pe branch 5 times, most recently from 58ac6a6 to 33588d7 Compare February 7, 2025 14:43
@martijnbastiaan martijnbastiaan force-pushed the hidde/switch-demo-pe branch 3 times, most recently from 3a58416 to a97b15d Compare February 10, 2025 10:49
@hiddemoll hiddemoll force-pushed the hidde/switch-demo-pe branch 2 times, most recently from 1e26ce4 to 31a33f9 Compare February 19, 2025 13:24
bittide/src/Bittide/SwitchDemoProcessingElement.hs Outdated Show resolved Hide resolved
bittide/src/Bittide/SwitchDemoProcessingElement.hs Outdated Show resolved Hide resolved
bittide/src/Bittide/SwitchDemoProcessingElement.hs Outdated Show resolved Hide resolved
bittide/src/Bittide/SwitchDemoProcessingElement.hs Outdated Show resolved Hide resolved
@hiddemoll hiddemoll force-pushed the hidde/switch-demo-pe branch 2 times, most recently from 53efd23 to 66bf8c2 Compare February 19, 2025 15:40
@hiddemoll hiddemoll marked this pull request as ready for review February 19, 2025 15:40
hiddemoll and others added 10 commits February 19, 2025 16:42
The incremented size of `readCycles` better matches the intended number
of tri-cycles the user writes in Rust, since this makes no sense to be
zero-indexed.
Having a lower max debug level for a debug build than a release build makes no
sense. The maximum level for a debug build is now 'trace' (the highest) and for
a release build it is 'info'.
The 'features' of the log module set in the respective Cargo.toml are global.
Adding other features in two Cargo.toml files in the same module gives a
duplicate features error.

Using the log module is slower than the previous method of a global constant
boolean `DEBUG_MODE` with if-statements around `uwrite` calls. Therefore, I had
to increase the values for when to start reading/writing in the switch demo PE
test.
By also returning the DNA, instead of only having it wishbone accessible, we can
also use the device DNA in hardware.
@hiddemoll hiddemoll force-pushed the hidde/switch-demo-pe branch from 66bf8c2 to 7165f89 Compare February 19, 2025 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants